-
Notifications
You must be signed in to change notification settings - Fork 406
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generate CycloneDX SBOMs using our own JSON generation #587
Conversation
Compare this output to that generated by the cyclonedx-gomod library proposed in #573: https://gist.github.com/imjasonh/31002f904a28166c9c8c9b6cedd07d96 |
Codecov Report
@@ Coverage Diff @@
## main #587 +/- ##
==========================================
- Coverage 48.92% 48.53% -0.40%
==========================================
Files 43 43
Lines 2228 2246 +18
==========================================
Hits 1090 1090
- Misses 954 972 +18
Partials 184 184
Continue to review full report at Codecov.
|
cc @puerco for thoughts |
I think it makes sense to write the small generator for both formats. Since the SBOM is only listing the dependencies for now, pulling in the full spdx/cycloneDX libraries seems like overkill. @imjasonh how much further do you see ko acquiring more sbom functions? I can think of three main areas which would be good to have, but maybe are out of scope for ko to do these:
|
Honestly, I hope we don't get too much more sophisticated than this, unless someone needs us to. The cyclonedx-gomod package is huge, but it's also very complete in its functionality. I'd like
If we include license information, I'd like to depend on some other service/metadata to look it up. cyclonedx-gomod's bloat largely comes from complex multilingual probabilistic license detection; I don't want to reimplement that, at all.
For both of these, I'm hoping to more completely implement cosign's stated SBOM spec (even though AFAIK nobody implements that today). This would let us report this SBOM only for the I don't want |
Awesome, I am currently working on that part. An important feature will be to be able to reference external documents from the deps SBOM generated by ko, I can add that feature for the next release. |
* Generate CycloneDX SBOMs using our own JSON generation * fix some errors * Add support to ko deps * Add e2e SBOM validation * ignore empty hashes (why are hashes empty?)
* Generate CycloneDX SBOMs using our own JSON generation * fix some errors * Add support to ko deps * Add e2e SBOM validation * ignore empty hashes (why are hashes empty?)
Example output: https://gist.github.com/imjasonh/16185e53200bed16470e5d27e17f261e 👀